Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Major] Update with string access and code refactory (#83) #93

Merged
merged 4 commits into from
Jan 26, 2024
Merged

[Major] Update with string access and code refactory (#83) #93

merged 4 commits into from
Jan 26, 2024

Conversation

frankaging
Copy link
Collaborator

Descriptions:
We add in string access of model component in the config for quick intervention without config. We now allow intervention instance to be initialized outside the config. We remove some dependency between a couple of util files and the config mapping.

Testing Done:

WARNING:root:The key is provided in the config. Assuming this is loaded from a pretrained module.
WARNING:root:Loading trainable intervention from intkey_layer.0.repr.block_output.unit.pos.nunit.1#0.bin.
WARNING:root:Loading trainable intervention from intkey_layer.1.repr.block_output.unit.pos.nunit.1#0.bin.
.Directory './test_output_dir_prefix-c8e990' created successfully.
.Directory './test_output_dir_prefix-890a9d' created successfully.
WARNING:root:Saving trainable intervention to intkey_layer.0.repr.block_output.unit.pos.nunit.1#0.bin.
WARNING:root:Saving trainable intervention to intkey_layer.1.repr.block_output.unit.pos.nunit.1#0.bin.
.Removing testing dir ./test_output_dir_prefix-25c2db
Removing testing dir ./test_output_dir_prefix-e57317
Removing testing dir ./test_output_dir_prefix-a8f096
Removing testing dir ./test_output_dir_prefix-c8e990
Removing testing dir ./test_output_dir_prefix-890a9d
......
----------------------------------------------------------------------
Ran 42 tests in 43.378s

OK

@frankaging frankaging self-assigned this Jan 26, 2024
@frankaging frankaging merged commit d1f7c58 into main Jan 26, 2024
1 check passed
@frankaging frankaging deleted the zen/stringaccess branch January 29, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@frankaging frankaging

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@frankaging